Skip to content

Conversation

Sukang1002
Copy link
Contributor

Copy link
Member

@yuxiqian yuxiqian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for @Sukang1002's update!

Kindly reminder: mvn verify -DskipTests could be executed locally to spot any code style violations in advance.

@Sukang1002
Copy link
Contributor Author

Thank you for your help.Please take the time to double-check again.

Copy link
Member

@yuxiqian yuxiqian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for @Sukang1002's nice work, LGTM.

+@lvyanquan as it affects all Debezium-based CDC sources.

Copy link
Contributor

@lvyanquan lvyanquan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1.

@lvyanquan lvyanquan merged commit bd24799 into apache:master Sep 15, 2025
60 of 61 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants